-
Notifications
You must be signed in to change notification settings - Fork 226
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
parallel run fixes and lint changes #809
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, but the new range syntax is not available in netstandard
var useAWS = Environment.GetEnvironmentVariable("AWS_ENDPOINT") is not null; | ||
if (Environment.GetEnvironmentVariable("SERVER_ENDPOINT") is not null) | ||
{ | ||
endPoint = Environment.GetEnvironmentVariable("SERVER_ENDPOINT"); | ||
var posColon = endPoint.LastIndexOf(':'); | ||
if (posColon != -1) | ||
{ | ||
port = int.Parse(endPoint.Substring(posColon + 1, endPoint.Length - posColon - 1), NumberStyles.Integer, | ||
port = int.Parse(endPoint.AsSpan(posColon + 1, endPoint.Length - posColon - 1), NumberStyles.Integer, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not available in netstandard2.0
CultureInfo.InvariantCulture); | ||
endPoint = endPoint.Substring(0, posColon); | ||
endPoint = endPoint[..posColon]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not available in netstandard2.0
@@ -2763,7 +2718,7 @@ internal static async Task ListenBucketNotificationsAsync_Test1(MinioClient mini | |||
// data. This is required to match and convert json data | |||
// "receivedJson" into class "ErrorResponse" | |||
var len = "{'Error':".Length; | |||
var trimmedFront = receivedJson.Substring(len); | |||
var trimmedFront = receivedJson[len..]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not available in netstandard2.0
@ebozduman would be good to get this merged asap so i can rebase my PR and the other 2 PR's can too |
I agree, but there are still some failures during build/functional tests. I think we need the flag to turn on/off parallel lrun flag, your suggestion, to be able to let those PRs pending to be merged just because the build tests fail when run in parallel, unrelated to the fixes/changes those PRs bring in. |
I reviewed the changes one more time and I think I need to revisit this one, rebase and resolve the conflicts and definitely merge at least 2, likely medium priority changes in, if not all. |
There is no need for this PR anymore. |
Fix for sporadic functional tests/ build test sporadic failures introduced after parallel run change.
Some a bit of lint changes are also included.